Need to take care here. What about changes to emcmot_hal_data->interp_feedrate?
Its defined as hal_float_t *interp_feedrate;
Not to mention my pending ones in #3995

    /* New Geometric Metadata Pins */
    hal_float_t *interp_arc_radius;
    hal_float_t *interp_arc_center_x;
    hal_float_t *interp_arc_center_y;
    hal_float_t *interp_arc_center_z;
    hal_float_t *interp_straight_heading;
    hal_float_t *interp_normal_heading;  
    hal_bit_t   *iscircle;

Which leads me to suggest another change to resolve the active_tags mess this touches which I was leaving unitil #3995 was committed.
add an extra float/double array to state_tags
double fields_extra_float[EX_FIELD_MAX_FIELDS]
Move all new pins in #3995 to this field and lock fields_float to the original fields (up to GM_FIELD_FLOAT_NAIVE_CAM_TOLERANCE) with an assert statement and apropiate comments in state_tags.h

The name hal_float_t is a complete misnomer and not a C-type float. The underlying type is actually a double.

The name "float" is used many places in linuxCNC, but most float types were promoted to double because you need the precision in the calculation. The variable names and defined did not change, only the underlying type.